[ErrorTracking] Update Issue side panel screenshots#34695
[ErrorTracking] Update Issue side panel screenshots#34695
Conversation
shiyueZ6
changed the title
jeff-morgan-dd
dismissed
their stale review
Re-checked guidelines for images on localized pages - some changes to the PR are recommended
jeff-morgan-dd
left a comment
jeff-morgan-dd
left a comment
There was a problem hiding this comment.
Hi Shiyue, thanks very much for this PR! I'm requesting some updates to adhere to best practices for changing images, which are intended to help prevent broken image links and ensure that the updated images display as intended.
Primarily, please keep the legacy image files in the directory, and add the new images with different file names (and update the image links as needed). Outdated images shouldn't be deleted at the time that they're replaced, which helps us account for delays in translation pipelines and caching issues. (We periodically run a script to remove images that aren't in use).
Please let me know if you have any questions! Thanks again for submitting the PR.
Preserve old image files for translation pipeline and caching, add updated screenshots with new filenames (e.g. -2, -3 suffixes), and update all markdown references accordingly. Co-authored-by: Cursor <cursoragent@cursor.com>
shiyueZ6
force-pushed
the
shiyue.zhang/update-error-tracking-panel-screenshot
branch
from
4b37512 to
41e85de
Compare
Co-authored-by: Cursor <cursoragent@cursor.com>
|
Changes since the last review:
@jeff-morgan-dd 🙇♀️ |
2 participants
What does this PR do? What is the motivation?
Updates screenshots in the Error Tracking section to reflect the current UI, and removes unused/obsolete image files.
Changes:
error-replay.png,error-tracking-overview.png,facets-panel.png,issue-correlation-*.png,issue-level-filters.png,issue-status-history.png,ownership-details.pngerror-tracking-overview.pnginstead of the olderror-tracking-overview-2.pngerror-tracking-overview-1.png,error-tracking-overview-2.png,issue-status-history-2.png,suspected-cause-filter.pngMerge instructions
Merge readiness:
Additional notes
Made with Cursor